Skip to content

Include github actions to multiline-regex-datacapture-filter plugin #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 15, 2025

Conversation

carlosrfranco
Copy link
Collaborator

No description provided.

@carlosrfranco carlosrfranco self-assigned this May 15, 2025
@edbaltra edbaltra requested review from edbaltra and Copilot May 15, 2025 15:17
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces automated GitHub Actions workflows for building and releasing the multiline-regex-datacapture-filter plugin.

  • Add a CI workflow (gradle.yml) to build the plugin, fetch tags, extract version, and upload the JAR as an artifact
  • Add a release workflow (release.yml) triggered on tag pushes to build, extract version, and publish a GitHub release with the plugin JAR

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/release.yml New workflow to create a GitHub release on tag push
.github/workflows/gradle.yml New Java CI workflow with build and artifact upload
Comments suppressed due to low confidence (2)

.github/workflows/gradle.yml:32

  • [nitpick] The artifact name Grails-Plugin-<version> is generic; consider renaming it to include the actual plugin identifier (e.g., multiline-regex-datacapture-filter-<version>) for clarity.
name: Grails-Plugin-${{ steps.get_version.outputs.VERSION }}

.github/workflows/release.yml:23

  • This workflow builds the project but does not run tests. Consider adding ./gradlew test before packaging to ensure no regressions slip through on release.
- name: Build with Gradle

Copy link

@edbaltra edbaltra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks great, I don’t have any comments to add.

@carlosrfranco carlosrfranco merged commit 67f8f4c into master May 15, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants